Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

test(cat-gateway): restore schemathesis tests #1434

Merged
merged 34 commits into from
Jan 8, 2025
Merged

test(cat-gateway): restore schemathesis tests #1434

merged 34 commits into from
Jan 8, 2025

Conversation

kukkok3
Copy link
Contributor

@kukkok3 kukkok3 commented Dec 24, 2024 •
edited
Loading

Description

This PR adds a catalyst gateway integration image with a running index db and a event db. It reenables the schemathesis tests only for the /api/v1/health/* endpoints. Before reenabling the other endpoints there are some errors to be fixed and bugs to be opened

Related Issue(s)

List the issue numbers related to this pull request.

e.g., Closes #1, Resolves #1 Fixes #1

Description of Changes

Provide a clear and concise description of what the pull request changes.

Breaking Changes

Describe any breaking changes and the impact.

Screenshots

If applicable, add screenshots to help explain your changes.

Related Pull Requests

If applicable, list any related pull requests.

e.g., #1, #1

Please confirm the following checks

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream module

@kukkok3 kukkok3 self-assigned this Dec 24, 2024
@kukkok3 kukkok3 linked an issue Dec 24, 2024 that may be closed by this pull request
🛠️ [TASK] : Investigate how to test the new cat gateway #912
Closed
@github-actions GitHub Actions
Copy link
Contributor

✅ Test Report | ${\color{lightgreen}Pass: 334/334}$ | ${\color{red}Fail: 0/334}$ |

@github-actions GitHub Actions
Copy link
Contributor

✅ Test Report | ${\color{lightgreen}Pass: 334/334}$ | ${\color{red}Fail: 0/334}$ |

@github-actions GitHub Actions
Copy link
Contributor

✅ Test Report | ${\color{lightgreen}Pass: 334/334}$ | ${\color{red}Fail: 0/334}$ |

@minikin minikin added the draft Draft label Dec 30, 2024
@kukkok3 kukkok3 marked this pull request as ready for review January 2, 2025 12:49
@kukkok3 kukkok3 requested review from emiride and oldGreg5 January 2, 2025 12:50
@kukkok3 kukkok3 added the review me PR is ready for review label Jan 2, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 2, 2025

✅ Test Report | ${\color{lightgreen}Pass: 370/370}$ | ${\color{red}Fail: 0/370}$ |

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 3, 2025

✅ Test Report | ${\color{lightgreen}Pass: 371/371}$ | ${\color{red}Fail: 0/371}$ |

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 6, 2025

✅ Test Report | ${\color{lightgreen}Pass: 371/371}$ | ${\color{red}Fail: 0/371}$ |

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 7, 2025

✅ Test Report | ${\color{lightgreen}Pass: 371/371}$ | ${\color{red}Fail: 0/371}$ |

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 7, 2025

✅ Test Report | ${\color{lightgreen}Pass: 370/370}$ | ${\color{red}Fail: 0/370}$ |

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 7, 2025

✅ Test Report | ${\color{lightgreen}Pass: 373/373}$ | ${\color{red}Fail: 0/373}$ |

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 7, 2025

✅ Test Report | ${\color{lightgreen}Pass: 373/373}$ | ${\color{red}Fail: 0/373}$ |

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 7, 2025

✅ Test Report | ${\color{lightgreen}Pass: 374/374}$ | ${\color{red}Fail: 0/374}$ |

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 8, 2025

✅ Test Report | ${\color{lightgreen}Pass: 374/374}$ | ${\color{red}Fail: 0/374}$ |

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 8, 2025

✅ Test Report | ${\color{lightgreen}Pass: 374/374}$ | ${\color{red}Fail: 0/374}$ |

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 8, 2025

✅ Test Report | ${\color{lightgreen}Pass: 377/377}$ | ${\color{red}Fail: 0/377}$ |

@kukkok3 kukkok3 requested a review from minikin January 8, 2025 10:54
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 8, 2025

✅ Test Report | ${\color{lightgreen}Pass: 377/377}$ | ${\color{red}Fail: 0/377}$ |

cong-or
cong-or approved these changes Jan 8, 2025
View reviewed changes
Copy link
Contributor

@cong-or cong-or left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 8, 2025

✅ Test Report | ${\color{lightgreen}Pass: 379/379}$ | ${\color{red}Fail: 0/379}$ |

@kukkok3 kukkok3 merged commit 4908eeb into main Jan 8, 2025
35 checks passed
@kukkok3 kukkok3 deleted the feat/cat-gateway-test-pacckage branch January 8, 2025 16:30
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 8, 2025

✅ Test Report | ${\color{lightgreen}Pass: 379/379}$ | ${\color{red}Fail: 0/379}$ |

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cong-or cong-or cong-or approved these changes

@stevenj stevenj Awaiting requested review from stevenj stevenj is a code owner

@Mr-Leshiy Mr-Leshiy Awaiting requested review from Mr-Leshiy Mr-Leshiy is a code owner

@saibatizoku saibatizoku Awaiting requested review from saibatizoku saibatizoku is a code owner

@bkioshn bkioshn Awaiting requested review from bkioshn bkioshn is a code owner

@stanislav-tkach stanislav-tkach Awaiting requested review from stanislav-tkach stanislav-tkach is a code owner

@emiride emiride Awaiting requested review from emiride

@oldGreg5 oldGreg5 Awaiting requested review from oldGreg5

@minikin minikin Awaiting requested review from minikin

Assignees

@kukkok3 kukkok3

Labels
review me PR is ready for review
Projects
Catalyst
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

🛠️ [TASK] : Investigate how to test the new cat gateway First
4 participants
@kukkok3 @cong-or @minikin @stevenj